Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add documentation for password change feature #1711

Merged
merged 9 commits into from
Jan 14, 2025

Conversation

latin-panda
Copy link
Contributor

Description

Documentation for password change feature.
medic/cht-core#9547

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

@latin-panda latin-panda changed the title password change feature chore: add documentation for password change feature Nov 15, 2024
@latin-panda
Copy link
Contributor Author

Screenshot 2024-11-15 at 3 32 12 PM

@latin-panda
Copy link
Contributor Author

@esthermmoturi and @n-orlowski please review, not sure if this feature will make it for 4.15 release so I've put it in 4.16 for now

@latin-panda
Copy link
Contributor Author

@n-orlowski I've applied the feedback, can you please review again?

Copy link
Contributor

@esthermmoturi esthermmoturi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few non-blocking suggestions.

@mrjones-plip mrjones-plip requested a review from jkuester January 14, 2025 18:22
@mrjones-plip
Copy link
Contributor

@jkuester - I added a minor change so didn't want to approve/merge this PR with out a quick spot check. Can you review and merge when you get a sec?

Copy link
Contributor

@jkuester jkuester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! One minor suggestion....

@mrjones-plip mrjones-plip merged commit 5505e99 into main Jan 14, 2025
2 checks passed
@mrjones-plip mrjones-plip deleted the new-password-change-feature-docs branch January 14, 2025 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants